✅ Code Review: APPROVED (with notes)

48/48 URI tests pass. Full suite: 1450 passed, 7 skipped — zero regressions.

Checklist walkthrough

1. Uri/Uriable structure and immutability ✓

Pattern correctly mirrors Str/Stringable:

• Uriable = fluent builder, wraps ParseResult + internal dict[str, list[str]] for query state
• Every mutator calls _rebuild() which instantiates a new Uriable — original is never touched
• TestFluencyChaining::test_immutability_across_chain confirms this structurally and behaviourally ✓

Auth preservation across mutations also verified manually:

Uri.of('https://user:pass@example.com:8080/api').with_host('other.com').get()
→ 'https://user:pass@other.com:8080/api'  ✓

2. support/init.py exports ✓

Both Uri and Uriable exported and listed in __all__. ✓

3. serve_command.py — urllib.parse → Uri.of() migration ℹ️ (N/A for this PR)

On this branch serve_command.py is the main version (pre-PR #111 rewrite). There is no urllib.parse import in it to replace. The _resolve_host_port + urlparse code lives only in PR #111's branch, which hasn't landed. The migration to Uri.of() belongs in a revised PR #111 — not here.

4. Tests in tests/utils/test_uri.py ✓

All 48 tests present. Fine location.

5. query().all(), .has(), .missing(), .decode() — ⚠️ Not implemented (see note)

6. No regressions ✓ (inferring the truncated checklist item)

Correctness spot-checks

Notes (non-blocking)

Note 1 — query() fluent methods not implemented (spec deviation)

PM checklist item 5 specified query().all(), .has(), .missing(), .decode(). The SE instead implemented query() → dict[str, list[str]] plus query_param(key, default) for single-key access.

The SE's approach is simpler and idiomatic Python — callers can use standard dict operations ("page" in uri.query(), uri.query().get("page")). However it differs from what was spec'd. Suggest a follow-up task if a fluent QueryString wrapper (.has(), .missing(), .decode()) is still wanted.

Note 2 — Schemaless URLs require a scheme

Uri.of('myapp.com:9000') returns host='', port=None because urlparse treats myapp.com as the scheme. This is standard urlparse behaviour — not a bug. If PR #111's _resolve_host_port is eventually rewritten to use Uri.of(), it must continue to prepend http:// for schemaless URLs before calling Uri.of(). Worth a one-liner note in the Uri.of() docstring.

Note 3 — _build_netloc uses Ellipsis sentinel with # type: ignore

Functional, but a named sentinel (_UNSET = object()) would be cleaner. Minor — not a blocker.

Verdict: APPROVED — safe to merge.